-
Notifications
You must be signed in to change notification settings - Fork 230
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(rules): add support for function declaration as test case #504
feat(rules): add support for function declaration as test case #504
Conversation
3eb69ad
to
52b05c8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cheers! I've commented on a few things to be actioned :)
6aac2a8
to
55caaed
Compare
@G-Rath Thanks for the fast review; i have now updated based on your requested changes and ensured coverage in tests meets 100% |
55caaed
to
0d11f9a
Compare
Add support for the following test file structure. ```js test('my test', myTest) function myTest() { expect(true).toBe(true) } ``` Methods that are directly referenced will be ananalyzed for the following rules `expect-expect` `no-if` `no-test-return-statement`, `no-try-expect`
0d11f9a
to
c4f7f87
Compare
🎉 This PR is included in version 23.4.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Add support for the following test file structure.
Methods that are directly referenced will be ananalyzed for the
following rules
expect-expect
no-if
no-test-return-statement
,no-try-expect